Skip to content

Order-edge claims: assert A4's admission-wall refusal, not the pre-A4 order-absent reason - #12939

Merged
briansrls merged 2 commits into
mainfrom
session/calm-fox-43-order-edge-claims
Oct 2, 2026
Merged

briansrls merged 2 commits into
mainfrom
session/calm-fox-43-order-edge-claims

Conversation

@gunbai-bot

@gunbai-bot gunbai-bot Bot commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

Fixes the three claims that #12514's merge-group differential found passing at base and failing at head, with #12770 (A4) between them. Measured red on main 224cdc4623f: each returned false.

Root cause: A4 moved where the refusal happens; no product behaviour broke

A4 (v2.std.node arrow_signature_order_conforms → arrow_domain_binds_a_name) makes an Arrow whose domain binds names but carries no order edge ill formed at Node admission. So v2.std.arrow_signature arrow_declared_parameter_order now answers ArrowParameterOrderMalformed for that shape, where before A4 it answered ArrowParameterOrderAbsent. Each claim read the old answer.

claim before A4 since A4
emit.produced_item_parameter_order.an_item_with_parameters_and_no_order_refuses_typed refused produced_decl_render_parameter_order_absent refused produced_decl_render_parameter_order_malformed
emit.realized_closure_parameter_order.a_closure_with_named_parameters_and_no_order_refuses_typed refused target_value_expr_reason_closure_parameter_order_absent refused target_value_expr_reason_closure_parameter_order_malformed
interface_summary.interface_summary_firewall.export_signature_host_seam_mints_no_order_edge the raw host Arrow read Absent it reads Malformed

In each case the input is still refused with a typed reason; the refusal now comes earlier, at the wall.

The re-statements (the discrimination is kept)

  • The two refusal claims first assert the route (the stripped Arrow reads ArrowParameterOrderMalformed at the admission wall), then the typed _malformed refusal. They still red if the renderer or closure realizer ever accepts the order-less form.
  • The host-seam claim keeps its intent: the v1 host seam mints no second order constructor, and A4 relies on that, since export_signature_facts is order-less on purpose and consumed only through export_signature_declared_facts. It now asserts this structurally: no edge carries arrow_signature_order_label. It also asserts the raw shape reads Malformed on its own, which is why the only consumer is the lift, the one constructor that makes it well formed. Its neighbour exported_signature_carries_named_binders_and_declared_order still pins that the lifted signature carries the declared order.

All three modules are green on a remote run (19 claims), including the unchanged neighbours.

🤖 Generated with Claude Code

gunbc-ci-auto-heal and others added 2 commits October 1, 2026 18:28
…ather than the pre-A4 order-absent reason

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@briansrls
briansrls added this pull request to the merge queue Oct 2, 2026
Merged via the queue into main with commit 6b85b9b Oct 2, 2026
4 checks passed
@briansrls
briansrls deleted the session/calm-fox-43-order-edge-claims branch October 2, 2026 03:47
gunbai-bot Bot pushed a commit that referenced this pull request Oct 2, 2026
…: #12960's requirements-edge claim builds and reads the core ArrowResourceRequirementsEdge; its member labels are Authored

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@briansrls
briansrls restored the session/calm-fox-43-order-edge-claims branch October 2, 2026 03:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant